Skip to content

Analyzer enhancements #696

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 31 commits into from
Jun 26, 2025
Merged

Analyzer enhancements #696

merged 31 commits into from
Jun 26, 2025

Conversation

halotukozak
Copy link
Member

No description provided.

@halotukozak halotukozak self-assigned this Apr 17, 2025
@halotukozak halotukozak requested a review from najder-k April 17, 2025 12:14
@halotukozak halotukozak force-pushed the analyzer-enhancements branch from c6f7fda to 746d402 Compare April 17, 2025 12:19
@halotukozak halotukozak requested a review from ddworak April 18, 2025 10:54
@halotukozak
Copy link
Member Author

@najder-k and @ddworak
can we proceed this PR (maybe we might find less busy reviewer)?

@ddworak
Copy link
Member

ddworak commented Jun 23, 2025

As mentioned when this started - I was mostly hoping for @najder-k to review this. Please decide together on the course of action here.

@ddworak ddworak removed their request for review June 23, 2025 06:52
Copy link
Contributor

@najder-k najder-k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Heck yeah brother, solid contrib 💪 just a few minor changes necessary

@halotukozak halotukozak requested a review from najder-k June 25, 2025 09:19
…o analyzer-enhancements

# Conflicts:
#	analyzer/src/test/scala/com/avsystem/commons/analyzer/ImplicitTypesTest.scala
…yzer-enhancements

# Conflicts:
#	analyzer/src/test/scala/com/avsystem/commons/analyzer/ImplicitParamDefaultsTest.scala
Copy link
Contributor

@najder-k najder-k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm w/ one nit

@halotukozak halotukozak merged commit 3f29426 into master Jun 26, 2025
7 checks passed
@halotukozak halotukozak deleted the analyzer-enhancements branch June 26, 2025 07:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants